[ADD, TWEAK] Protolathes and Space Forge - #11
Conversation
Summary of ChangesHello @BL02DL, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! Этот PR обновляет визуальное представление протолазеров в игре, вводя новые, специализированные спрайты для машин различных отделов (инженерного, грузового, научного) и добавляя совершенно новый сервисный протолазер. Изменения также включают обновление анимации, отображаемой во время работы этих устройств, что способствует более наглядному игровому процессу. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
RSI Diff Bot; head commit 9e324ad merging into efad73c Resources/Textures/_LP/Structures/Machines/SpaceForge.rsi
Resources/Textures/_LP/Structures/Machines/cargo_protolathe.rsi
Resources/Textures/_LP/Structures/Machines/engi_protolathe.rsi
Resources/Textures/_LP/Structures/Machines/rnd_protolathe.rsi
Resources/Textures/_LP/Structures/Machines/serv_protolathe.rsi
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
Этот PR обновляет спрайты и анимации для различных протолазерных станков, а также добавляет новый сервисный протолазер. В целом, изменения выглядят хорошо, но есть несколько моментов, которые требуют внимания. Я оставил комментарии по поводу временных правок в коде, несоответствий в именовании новой сущности и некоторых других мелких улучшений. Пожалуйста, ознакомьтесь с ними.
| name: engineering techfab | ||
| description: Prints equipment and circuitboards for use by engineering department. |
There was a problem hiding this comment.
Название (name) и описание (description) этого прототипа, похоже, скопированы из EngineeringTechFab. Однако id (LPPServiceProtolathe) и спрайт (serv_protolathe.rsi) указывают на то, что это сервисный протолазер. Для ясности и консистентности стоит обновить name и description, чтобы они соответствовали назначению объекта.
name: service protolathe
description: Prints equipment and circuitboards for use by the service department.| # LP Edit Start | ||
| sprite: _LP/Structures/Machines/engi_protolathe.rsi | ||
| snapCardinals: true | ||
| # LP Edit End |
There was a problem hiding this comment.
В этом файле в нескольких местах остались временные комментарии, такие как # LP Edit Start/End и # LP Edit (здесь, а также на строках 47, 103-106, 122, 173-176, 192). Их следует удалить для чистоты кода.
sprite: _LP/Structures/Machines/engi_protolathe.rsi
snapCardinals: true| - state: icon | ||
| map: ["enum.LatheVisualLayers.IsRunning"] | ||
| - state: engi | ||
| # - state: engi LP Edit |
There was a problem hiding this comment.
Закомментированные слои, такие как engi на этой строке (а также cargo на строке 110 и sci на строке 180), следует удалить, если они больше не нужны. Оставление закомментированного кода без объяснений затрудняет поддержку.
| - type: Lathe | ||
| idleState: icon | ||
| runningState: icon | ||
| runningState: building # LP Edit |
There was a problem hiding this comment.
Временный комментарий # LP Edit следует удалить. Подобные комментарии также присутствуют на строках 122 и 192.
runningState: building| - UpgradeKits | ||
| # CorvaxGoob Packs | ||
| - WorkBeltsSharedDynamic # WorkBelts-Techfab | ||
| - WokrBeltsEngDynamic # WorkBelts-Techfab |
There was a problem hiding this comment.
Здесь опечатка в названии пакета: WokrBeltsEngDynamic. Вероятно, должно быть WorkBeltsEngDynamic.
- WorkBeltsEngDynamic # WorkBelts-Techfab| tags: | ||
| - Sheet | ||
| - RawMaterial | ||
| - Ingot No newline at end of file |
There was a problem hiding this comment.
Файл заканчивается без перевода строки. Рекомендуется добавлять пустую строку в конец файла для соответствия стандартам и лучшей совместимости с инструментами разработки.
2 participants
Описание PR
Изменение спрайтов техфабов, изменение их перевода на протолаты, добавление сервисного протолата, а также космической кузни. Космическая кузница: аппарат создающий скафандры разных типов которые теперь изучаются в РнД, она разделяется на два типа: Гражданская и Командная где каждая имеет разный набор скафандров, первую можно только изучить и после сделать, вторая имеется со старта смены на мостике и также плата дополнительная будет в капитана, однако вторая плата является Хай риск целью.
Почему / Баланс
Изменения основывающиеся на Space Station 13, где у каждого отдела были протолаты, а не техфабы, за исключением мед отдела и службы безопасности. Также космическая кузница была добавлена для того, чтобы в игре можно было создавать скафандры.
Технические детали
Плата кузницы командого типа находящаяся у капитана ровно та же что и при разборе кузницы со старта смены.
ToDo
Медиа
Требования
Критические изменения
Список изменений
🆑 BL02DL